New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Collect fields for modules and metricsets #1493
Conversation
ruflin
commented
Apr 26, 2016
- Add python script to collect fields.yml files
- Update templates and docs
- Add fields.yml for each module
@@ -4,19 +4,6 @@ | |||
"_all": { | |||
"norms": false | |||
}, | |||
"dynamic_templates": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tsg Did you add this part manually that is is now missing here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tsg Ignore this one, found the issue
8ba5afa
to
4ff96f6
Compare
@@ -0,0 +1,36 @@ | |||
#!/usr/bin/env bash | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file isn't used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ups, that was a second try :-( forgot to remove it :-(
4ff96f6
to
0010905
Compare
print fields_yml | ||
|
||
|
||
collect() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not much a python programmer, but isn't the convention to use https://docs.python.org/2/library/__main__.html for the entrypoint.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's good practice to do that because then you can import functions/classes from other files. Can still be added when actually needed.
* Add python script to collect fields.yml files * Update templates and docs * Add fields.yml for each module
0010905
to
7bdf627
Compare
@andrewkroh Thanks for the review. Version with changes pushed. |
LGTM |